Greptile Summary

Major test suite refactoring that consolidates CDP-based tests into a unified structure. The PR removes ~5,000+ lines of old controller-based and CDP test files, replacing them with ~1,000 lines of new streamlined CDP tests using a shared withBrowser helper.

Key changes:

• New test infrastructure: Created with-browser.ts helper that provides cached CDP browser connection with mutex for concurrent test execution
• Test consolidation: Removed separate tests/tools/cdp/ and tests/tools/controller/ directories, replaced with unified tests/tools/ tests
• Package scripts updated: Consolidated test commands from separate test:cdp and test:controller into single test:tools command
• Bug fix in browser.ts: Added retry logic (10 attempts, 100ms delay) when fetching tab info after creation to handle race conditions
• Lint warnings addressed: Added biome-ignore comments with TODO notes for excessive complexity and any types in GraphQL code
• Minor cleanup: Updated GitHub Actions workflow, bumped Biome schema version

The refactoring maintains test coverage while significantly reducing code duplication and improving test helper reusability.

Confidence Score: 4/5

• Safe to merge with minor cleanup recommended for commented code
• Large refactoring successfully consolidates test infrastructure and reduces duplication. The new withBrowser helper and retry logic in browser.ts improve test reliability. Minor issues include leftover commented-out code and silent error swallowing in retry loop. The TODO comments acknowledge technical debt appropriately.
• apps/agent/web-ext.config.ts needs commented code removed

Important Files Changed

_{Last reviewed commit: 1d4763e}